Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] Add client manager in server #383

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

DavdGao
Copy link
Collaborator

@DavdGao DavdGao commented Sep 22, 2022

Target

  • We are planning to add a consultation stage before training for differential privacy and other algorithms.
  • We decompose it into several PRs to avoid large modification at one time.

What's in this PR

  • This PR is to add client manager in server, which is responsible for
    • Maintain the status of clients, e.g., working, idle or unseen (in training)
    • Manage the clients, including registration, client-related information storage (client_resource, [TODO]join_in_info)

Why

  • Currently, the client management is too decentralized. For example,
    • self.join_in_info stores the client information,
    • self.id_unseen_client records the unseen clients during training and
    • self.sampler records the status of the clients (Maybe sampler should be in charge of the client status)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant